Skip to content

Conversation

@rriveramcrus
Copy link
Contributor

The STM32F401XE and STM32F401XD family of devices are identical except for a difference in the flash storage size. Let's reuse the F401XE HAL symbol for the newly introduced F401XD variants.

The STM32F401XE and STM32F401XD family of devices are identical except
for a difference in the flash storage size. Let's reuse the F401XE HAL
symbol for the newly introduced F401XD variants.

Signed-off-by: Ricardo Rivera-Matos <[email protected]>
@sonarqubecloud
Copy link

config SOC
default "stm32f401xc" if SOC_STM32F401XC
default "stm32f401xd" if SOC_STM32F401XD
default "stm32f401xe" if SOC_STM32F401XD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, the name must match the symbol

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, the name must match the symbol

@nordicjm can you elaborate on this comment? That doesn't seem to be explicitly true for, let's say, soc/nordic/nrf52/Kconfig.soc.

zephyrproject-rtos/hal_stm32#276
Some additional context around this draft PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The name must match the symbol

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Jul 19, 2025
@github-actions github-actions bot closed this Aug 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants